-
Notifications
You must be signed in to change notification settings - Fork 26.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disable Aggregation by default #12184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need fix test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
test need be solve
done |
Kudos, SonarCloud Quality Gate passed! |
Codecov Report
@@ Coverage Diff @@
## 3.2 #12184 +/- ##
============================================
- Coverage 69.94% 69.54% -0.40%
+ Complexity 340 2 -338
============================================
Files 3404 1593 -1811
Lines 159368 65437 -93931
Branches 26552 9526 -17026
============================================
- Hits 111464 45511 -65953
+ Misses 38218 15532 -22686
+ Partials 9686 4394 -5292 see 1851 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What is the purpose of the change
Brief changelog
Verifying this change
Checklist